576884ec10b4a46b243b8138926c0fc650bcdaba,api/src/org/apache/cloudstack/api/command/admin/ldap/LDAPConfigCmd.java,LDAPConfigCmd,execute,#,156

Before Change


                  // return the existing conf
                  LDAPConfigCmd cmd = _configService.listLDAPConfig(this);
                  LDAPConfigResponse lr = _responseGenerator.createLDAPConfigResponse(cmd.getHostname(), cmd.getPort(), cmd.getUseSSL(), cmd.getQueryFilter(), cmd.getSearchBase(), cmd.getBindDN());
                  lr.setResponseName(getCommandName());
                  this.setResponseObject(lr);
              }
              else if (getHostname()==null || getSearchBase() == null || getQueryFilter() == null) {
                  throw new InvalidParameterValueException("You need to provide hostname, searchbase and queryfilter to configure your LDAP server");

After Change


              if (getListAll()){
                  // return the existing conf
                  LDAPConfigCmd cmd = _configService.listLDAPConfig(this);
                  ListResponse<LDAPConfigResponse> response = new ListResponse<LDAPConfigResponse>();
                  List<LDAPConfigResponse> responses = new ArrayList<LDAPConfigResponse>();

                  if(!cmd.getHostname().equals("")) {
                  	responses.add(_responseGenerator.createLDAPConfigResponse(cmd.getHostname(), cmd.getPort(), cmd.getUseSSL(), cmd.getQueryFilter(), cmd.getSearchBase(), cmd.getBindDN()));
                  }
                  
                  response.setResponses(responses);
                  response.setResponseName(getCommandName());
                  this.setResponseObject(response);
              }
              else if (getHostname()==null || getSearchBase() == null || getQueryFilter() == null) {
                  throw new InvalidParameterValueException("You need to provide hostname, searchbase and queryfilter to configure your LDAP server");